-
Notifications
You must be signed in to change notification settings - Fork 49
docs: add user permission docs #2147
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Signed-off-by: Dennis Zhuang <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds comprehensive documentation for user permission management in GreptimeDB's static authentication, expanding beyond basic username/password configuration to include role-based access control.
Key changes:
- Adds permission mode configuration (readonly, writeonly, readwrite) with syntax examples
- Documents dynamic file reloading capability using
watch_file_user_provider
- Provides detailed configuration examples with mixed permission scenarios
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
File | Description |
---|---|
docs/user-guide/deployments-administration/authentication/static.md | English documentation with permission modes, dynamic reloading, and comprehensive examples |
i18n/zh/docusaurus-plugin-content-docs/current/user-guide/deployments-administration/authentication/static.md | Chinese translation of the same permission documentation updates |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Deploying greptime-docs with
|
Latest commit: |
c0b14e3
|
Status: | ✅ Deploy successful! |
Preview URL: | https://4d7f39e7.greptime-docs.pages.dev |
Branch Preview URL: | https://feature-static-user-permissi.greptime-docs.pages.dev |
...s-plugin-content-docs/current/user-guide/deployments-administration/authentication/static.md
Outdated
Show resolved
Hide resolved
...s-plugin-content-docs/current/user-guide/deployments-administration/authentication/static.md
Show resolved
Hide resolved
docs/user-guide/deployments-administration/authentication/static.md
Outdated
Show resolved
Hide resolved
...s-plugin-content-docs/current/user-guide/deployments-administration/authentication/static.md
Outdated
Show resolved
Hide resolved
Signed-off-by: Dennis Zhuang <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
...s-plugin-content-docs/current/user-guide/deployments-administration/authentication/static.md
Outdated
Show resolved
Hide resolved
...s-plugin-content-docs/current/user-guide/deployments-administration/authentication/static.md
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Signed-off-by: Dennis Zhuang <[email protected]>
Signed-off-by: Dennis Zhuang <[email protected]>
Signed-off-by: Dennis Zhuang <[email protected]>
What's Changed in this PR
Describe the change in this PR
Close #2144
The k8s cluster needs to support the permission mode too. cc @zyy17
Checklist
sidebars.ts
matches the current document structure when you changed the document structure.